镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

[net6] Bump DotNetPreview version to 6.0.100-preview.1.21109.8 - #5610

Merged
dellis1972 merged 12 commits into
dotnet:masterfrom
radekdoulik:pr-bump-net6-preview.1.21109.8
Mar 1, 2021
Merged

dellis1972 merged 12 commits into
dotnet:masterfrom
radekdoulik:pr-bump-net6-preview.1.21109.8

Conversation

@radekdoulik

@radekdoulik radekdoulik commented Feb 10, 2021 •

Copy link
Copy Markdown
Member

Bump DotNetPreview version to 6.0.100-preview.1.21109.8

Add linker reflection based workaround, till
the new API is ready to use.
It forces the ILLink to process all the referenced assemblies in our steps,
without it, no assemblies are processed before MarkStep.

Updated apk size reference files. The NET6 linker has more optimizations
enabled, assembly sizes decreased about 0.5% and native shared libraries
are smaller as well - about 0.7%.

Comparison of BuildReleaseArm64 apk test sizes before/after

Simple XA NET6

Summary:
  +           0 Other entries 0.00% (of 52,514)
  +           0 Dalvik executables 0.00% (of 316,876)
  -       3,237 Assemblies -0.47% (of 690,451)
  -      34,992 Shared libraries -0.67% (of 5,202,312)
  -       7,680 Uncompressed assemblies -0.57% (of 1,357,824)
  -      10,670 Package size difference -0.36% (of 2,927,336)

XForms XA NET6

Summary:
  +           0 Other entries 0.00% (of 750,947)
  +           0 Dalvik executables 0.00% (of 3,454,884)
  -      29,117 Assemblies -0.51% (of 5,696,363)
  -      34,992 Shared libraries -0.66% (of 5,277,504)
  -      66,048 Uncompressed assemblies -0.51% (of 12,839,424)
  -      36,556 Package size difference -0.37% (of 9,899,330)

Simple XA legacy monodroid

Summary:
  +           0 Other entries 0.00% (of 52,485)
  +           0 Dalvik executables 0.00% (of 316,848)
  +           1 Assemblies 0.00% (of 1,153,531)
  +           0 Shared libraries 0.00% (of 10,510,880)
  +           0 Uncompressed assemblies 0.00% (of 2,765,312)
  -         116 Package size difference -0.00% (of 5,001,423)

XForms XA legacy monodroid

Summary:
  +           0 Other entries 0.00% (of 750,918)
  +           0 Dalvik executables 0.00% (of 3,454,264)
  -           2 Assemblies -0.00% (of 4,674,321)
  +           0 Shared libraries 0.00% (of 10,582,264)
  +           0 Uncompressed assemblies 0.00% (of 11,293,696)
  -          12 Package size difference -0.00% (of 10,476,653)

@radekdoulik

Copy link
Copy Markdown
Member Author

Testing on CI, looks like illink lost some steps.

@radekdoulik
radekdoulik force-pushed the pr-bump-net6-preview.1.21109.8 branch from d436b59 to 24a48bf Compare February 19, 2021 18:59
The linker doesn't load all assemblies anymore. We depend on processing
of all assemblies before `MarkStep`.

Use reflection as temporary workaround to overcome that, until we have
new API in the linker available to handle our processing as part
of `MarkStep`.
This also gets rid of typo in previous commit, which caused problems
in XF test.
@radekdoulik
radekdoulik force-pushed the pr-bump-net6-preview.1.21109.8 branch from 24a48bf to add54f8 Compare February 25, 2021 11:53
@radekdoulik radekdoulik changed the title Bump DotNetPreview version to 6.0.100-preview.1.21109.8 [net6] Bump DotNetPreview version to 6.0.100-preview.1.21109.8 Feb 25, 2021
@radekdoulik
radekdoulik marked this pull request as ready for review February 25, 2021 14:53
@jonathanpeppers

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@jonathanpeppers

Copy link
Copy Markdown
Member

@radekdoulik maybe we should use the same version as macios:

@radekdoulik

Copy link
Copy Markdown
Member Author

I would like to finish it with 6.0.100-preview.1.21109.8 so that I don't have to remeasure it and fix possible new issues. Will try the preview2 in new PR after that one. Hopefully it will be less bumpy than this one.

@dellis1972
dellis1972 merged commit dcd1cb7 into dotnet:master Mar 1, 2021
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants